Skip to content

[codex] fix nested spec parsing in CLI#1

Merged
auscaster merged 1 commit intomainfrom
codex/trellis-nested-spec-parsing
Mar 31, 2026
Merged

[codex] fix nested spec parsing in CLI#1
auscaster merged 1 commit intomainfrom
codex/trellis-nested-spec-parsing

Conversation

@auscaster
Copy link
Copy Markdown
Contributor

What changed

  • taught trellis complete to recognize nested acceptance result blocks written as result.status
  • taught trellis complete and trellis report to read nested self_eval.total, including decimal totals
  • fixed trellis status to count phase statuses only from the phases block instead of subtracting the top-level spec status
  • added smoke coverage for the nested-result, decimal-self-eval, and phase-count regressions

Why

trellis exec already records nested result blocks, but complete and report were still looking for older flat YAML shapes. That produced false warnings about missing exec results and self-eval scores, and status could undercount completed phases on archived specs.

Validation

  • python3 -m py_compile cli/trellis
  • CLI_ROOT=/home/kam/dev/trellis/cli bash tests/review_gate_smoke.sh all
  • live check against the archived Nitrosend spec

Handle nested acceptance result blocks and self_eval totals in complete/report. Also count phase statuses from the phases block only and cover the regressions with smoke tests.
@auscaster auscaster merged commit 008c575 into main Mar 31, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant